Skip to content

Issue #446: Admin JS broken after fresh install#451

Open
alexmerlin wants to merge 7 commits into
7.0from
7.0-clean-packages
Open

Issue #446: Admin JS broken after fresh install#451
alexmerlin wants to merge 7 commits into
7.0from
7.0-clean-packages

Conversation

@alexmerlin

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

Qodana for JS

1 new problem were found

Inspection name Severity Problems
Void function return value used 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@github-actions

Copy link
Copy Markdown

Qodana for JS

1 new problem were found

Inspection name Severity Problems
Void function return value used 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@alexmerlin alexmerlin linked an issue Jun 17, 2026 that may be closed by this pull request
@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.53%. Comparing base (9bc6c43) to head (6159035).
⚠️ Report is 6 commits behind head on 7.0.

Additional details and impacted files
@@            Coverage Diff            @@
##                7.0     #451   +/-   ##
=========================================
  Coverage     38.53%   38.53%           
  Complexity      925      925           
=========================================
  Files           169      169           
  Lines          3768     3768           
=========================================
  Hits           1452     1452           
  Misses         2316     2316           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@alexmerlin

Copy link
Copy Markdown
Member Author

@GeorgeCatalinStan I made a couple of fixes/updates.
One last thing left to fix: opening a modal on the /admin/list-admin and /user/list-user pages fails without an error.

@alexmerlin

alexmerlin commented Jun 18, 2026

Copy link
Copy Markdown
Member Author

@GeorgeCatalinStan Everything seems to be working as intended. 👍

I saw that you restored the jquery-sparkline and perfect-scrollbar packages.
Are they required by the app to function properly?

George and others added 7 commits June 18, 2026 07:06
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
Signed-off-by: alexmerlin <alex.merlin.1985@gmail.com>
@alexmerlin alexmerlin force-pushed the 7.0-clean-packages branch from 10a5393 to 6159035 Compare June 18, 2026 04:06
@alexmerlin

Copy link
Copy Markdown
Member Author

@GeorgeCatalinStan

Please note that:

  • I updated your branch by merging branch 7.0 into it (bringing the Build Assets CI workflow for dependency checks)
  • I fixed the DCO check (signature)

Make sure that you update your local branch if you need to make further adjustments to this branch/PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin JS broken after fresh install

1 participant